Skip to content

[Feature] 메인 뷰 UI 작업#4

Merged
Peter1119 merged 13 commits intodevelopfrom
feature/main_view
Aug 12, 2025
Merged

[Feature] 메인 뷰 UI 작업#4
Peter1119 merged 13 commits intodevelopfrom
feature/main_view

Conversation

@Peter1119
Copy link
Collaborator

🔗 관련 이슈

  • 관련 이슈: #

✨ 작업 내용

  • 메인 화면 리스트 row view 및 ui 작업

📸 Showcase

변경 전 변경 후
이미지 이미지

📌 이미지가 없다면 이 섹션은 생략해도 됩니다.


📝 참고 사항

Motivation 🥳 (코드를 추가/변경하게 된 이유)

Key Changes 🔥 (주요 구현/변경 사항)

  • 필요한 이미지 asset 넣었습니다. svg로 넣는 것을 선호해서 svg로 넣었습니다.
  • text primary, text secondary 색상을 피그마 디자인에 맞게 수정했습니다.
  • font의 enum 케이스를 소문자로 바꿨습니다.

To Reviewers 🙏 (리뷰어에게 전달하고 싶은 말)

  • 우선 모델에 대한 정의가 되어있지 않아서 임의로 row model을 넣었고 필요한 데이터에 대한 정의가 필요해 보입니다.

Reference 🔗

Close Issues 🔒 (닫을 Issue)

Close #No.

Checklist

  • 브랜치를 가져와 작업한 경우 이전 브랜치에 PR을 보냈는지 확인
  • 빌드를 위해 SceneDelegate 수정한 것 PR로 올리지 않았는지 확인
  • 필요없는 주석, 프린트문 제거했는지 확인
  • 컨벤션 지켰는지 확인
  • final, private 제대로 넣었는지 확인
  • 다양한 디바이스에 레이아웃이 대응되는지 확인
    • iPhone SE
    • iPhone 13
    • iPhone 13 Pro Max

@auto-assign auto-assign bot requested a review from minneee August 12, 2025 00:12
@Peter1119 Peter1119 changed the title Feature/main view [Feature] 메인 뷰 UI 작업 Aug 12, 2025
Copy link
Collaborator

@minneee minneee left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

고생하셨습니다~!

}
.frame(maxWidth: .infinity)
.padding(16)
.background(Color(hex: "E9EBEF"))
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

디자인 시스템에서 컬러 지정해서 사용하는게 아니라 hex로 컬러를 지정해서 사용하신 이유가 있으신가요??

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

동일한 컬러가 안나와있고 이름 명칭이 정의되어 있지 않아서 우선 이렇게 넣었어요
이건 맞춰가면 될 것 같습니다 !

@Roy-wonji Roy-wonji self-requested a review August 12, 2025 01:47
Copy link
Contributor

@Roy-wonji Roy-wonji left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

수고 하셨습니다 코멘트 달아들린거 확인해주시면 될거 같습니다


private let item: TeamExploreItem

private var title: String {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Peter1119 이것도 enum 안에 넣는 건 어떨까요 ? , 색상이랑 이미지도 넣어주시는거 어떨까요 ?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

넵 저도 고민했는데 같이 묶일 데이터 인가 ? 혹은
공유될 자원인가? 이런 고민이 있어서 우선 분리 시켰는데
합쳐도 괜찮을 것같습니다 ~!

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

이미지 추가했습니다! 확인부탁드립니다 ~!

@Peter1119 Peter1119 merged commit 8bda99a into develop Aug 12, 2025
@Peter1119 Peter1119 deleted the feature/main_view branch August 12, 2025 03:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants